Application: Do not reuse Ctrl+F1 for help-overlay
authorDaniel Boles <dboles@src.gnome.org>
Fri, 24 Aug 2018 11:51:21 +0000 (14:51 +0300)
committerRobert Roth <robert.roth.off@gmail.com>
Fri, 24 Aug 2018 11:51:21 +0000 (14:51 +0300)
commit4b96e7fc789b3b4542bfd50125e315e731fa80d5
tree8a22f4a4cd8aa0796c06432bdf23a1c3932fdde1
parent9e0f43e8cba45015487c692058c002f56087b9b8
Application: Do not reuse Ctrl+F1 for help-overlay

It just gets overridden to do something else, as the 1st GtkWidget’s
class_init() adds it as a binding to toggle tooltips with the keyboard.
The last entity to hook it (& return TRUE) is the only one who gets it.

So, worse: If users needed to manually set accels for a ShortcutsWindow,
coincidentally waited until after the 1st GtkWidget init() to do it, &
copied what GtkApplication said in an attempt to be good & consistent —
they inadvertently broke the keyboard tooltips (except for users with a
GDK_KEY_KP_F1 – whatever that is) as their handler blocked GtkWidget’s.

So, one side has to drop this accelerator, and it seems clear that being
able to open tooltips with the keyboard is more important than having a
second accel for the help-overlay. We can make up a replacement later.

https://gitlab.gnome.org/GNOME/gtk/issues/626
gtk/gtkapplication.c